Skip to content

Multiscale outputs struct rework#134

Merged
Samuel-amap merged 25 commits into
devfrom
multiscale_outputs_struct_rework
Apr 28, 2025
Merged

Multiscale outputs struct rework#134
Samuel-amap merged 25 commits into
devfrom
multiscale_outputs_struct_rework

Conversation

@Samuel-amap
Copy link
Copy Markdown
Collaborator

Outputs reworked to a Dict{String, Vector{NamedTuple}} structure.
Fewer memory allocations, faster to convert to DataFrame, more compact

…sage gain on the XPalm side). Currently deactivated. Some details will need fixing, many tests will be broken (for both PSE and XPalm), documentation not up-to-date
…tputs. Should probably document some of the differences with single-scale conversion (CSV friendly, no refvectors)
… approach as the previous output preallocation function), to fix test-mtg in PBP, the only test making use of MutableNodeMTG
…s now pass for PSE, XPalm and PBP. Performance should be adequate, to be confirmed, possible light loss for XPalm. Benchmarks and documentation still need work.
…nstream folder, let's uh, remove it from the repo
…e got renamed from XPalm.jl to XPalm ? I had to tinker with my local repo at some point, I think that's why)
@Samuel-amap Samuel-amap linked an issue Apr 14, 2025 that may be closed by this pull request
@Samuel-amap Samuel-amap changed the base branch from main to dev April 28, 2025 13:36
@Samuel-amap Samuel-amap merged commit 3f088d1 into dev Apr 28, 2025
0 of 22 checks passed
VEZY referenced this pull request in VEZY/PlantBiophysics.jl Jul 10, 2025
@Samuel-amap Samuel-amap deleted the multiscale_outputs_struct_rework branch September 19, 2025 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change default output structure

1 participant